Skip to content

Follow-up: XLSB Support #31215

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 35 commits into from
Jan 22, 2020
Merged

Follow-up: XLSB Support #31215

merged 35 commits into from
Jan 22, 2020

Conversation

Rik-de-Kort
Copy link
Contributor

Updated min version for pyxlsb to 1.0.6 and removed the now unnecessary xfail for read from url. Turns out the other xfail is actually unnecessary so I've removed it.

Question: is my name supposed to show up in the contributors list?

@WillAyd WillAyd added Docs IO Excel read_excel, to_excel labels Jan 22, 2020
@WillAyd WillAyd added this to the 1.0.0 milestone Jan 22, 2020
@WillAyd WillAyd merged commit 774498b into pandas-dev:master Jan 22, 2020
meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Jan 22, 2020
@WillAyd
Copy link
Member

WillAyd commented Jan 22, 2020

@Rik-de-Kort yea your name should auto-generate as a contributor when we release

Thanks!

TomAugspurger pushed a commit that referenced this pull request Jan 22, 2020
@Rik-de-Kort
Copy link
Contributor Author

@Rik-de-Kort yea your name should auto-generate as a contributor when we release

Thanks!

Excellent, and you're welcome. Maybe next patch we go a bit quicker, eh? :P

@Rik-de-Kort
Copy link
Contributor Author

@Rik-de-Kort yea your name should auto-generate as a contributor when we release

Thanks!

@WillAyd Happy to see the release is out, with my code :D However, my name does not appear to have been auto-generated in the contributors list. I do have the tag here though. Any ideas?

@WillAyd
Copy link
Member

WillAyd commented Feb 1, 2020

Hmm @TomAugspurger any ideas on the contributor piece? Original PR of #29836 was part of 1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs IO Excel read_excel, to_excel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement: XLSB support in read_excel()
2 participants